Skip to content

Conversation

@rnc
Copy link
Collaborator

@rnc rnc commented Nov 6, 2024

This also moves the servers block from the build stage (as we don't need auth for downloads) to the deployment stage (and the deployment stage transparently handles that if ACCESS_TOKEN is set)

@openshift-ci openshift-ci bot requested a review from vibe13 November 6, 2024 08:18
@codecov
Copy link

codecov bot commented Nov 6, 2024

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Project coverage is 43.78%. Comparing base (70a533a) to head (95b284b).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...m/redhat/hacbs/container/deploy/DeployCommand.java 0.00% 6 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2261      +/-   ##
============================================
+ Coverage     43.11%   43.78%   +0.67%     
  Complexity      803      803              
============================================
  Files           241      275      +34     
  Lines          8359    13252    +4893     
  Branches       1345     1347       +2     
============================================
+ Hits           3604     5803    +2199     
- Misses         4355     6850    +2495     
- Partials        400      599     +199     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@openshift-ci openshift-ci bot added the lgtm label Nov 6, 2024
@rnc rnc merged commit 1b76cdb into redhat-appstudio:main Nov 6, 2024
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants